Skip to content

[feat] ai-security-guard refactor & support checking multimoadl input #3075

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

[feat] ai-security-guard refactor & support checking multimoadl input #3075

wants to merge 11 commits into from

Conversation

@rinfx
Copy link
Collaborator

@rinfx rinfx commented Oct 31, 2025 •
edited by github-actions bot
Loading

Ⅰ. Describe what this PR did

  1. 对代码进行重构，进行了模块化拆分，以适应从文本处理到多模态处理的转变
  2. 文本生成场景支持多模态输入检测

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

Ⅰ. Describe what this PR did

  1. The code was restructured and modularized to adapt to the transition from text processing to multi-modal processing.
  2. Text generation scenarios support multi-modal input detection

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

rinfx added 5 commits October 29, 2025 13:56
@rinfx
v1
8dda27d 
Change-Id: I2757730d5c17a517b791eff6ac10e06bf4a12c8d
Co-developed-by: Cursor <[email protected]>
@rinfx
refactor
ce58aca 
Change-Id: I430d34a44e6f6a034bcad2769fef60b017ff58cb
Co-developed-by: Cursor <[email protected]>
@rinfx
bugfix
12f4d71 
Change-Id: Id6629e75209484b7d28c4c9dc347a6cd057a4373
Co-developed-by: Cursor <[email protected]>
@rinfx
update
7b0d93e 
Change-Id: I3e202c165cc02f8a45113415bb4b1607f1cc02a0
Co-developed-by: Cursor <[email protected]>
@rinfx
remove some comments
a847bbd 
Change-Id: I35a293d5f624677f51175feffa1651bfe45cabaa
Co-developed-by: Cursor <[email protected]>
@rinfx rinfx changed the title Ai sec refactor [feat] ai-security-guard refactor & support checking multimoadl input Oct 31, 2025
@rinfx
update
bd97257 
Change-Id: Ie427cff9cd0ba14a7078c6091a4cb4adbaa66999
Co-developed-by: Cursor <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Oct 31, 2025 •
edited
Loading

Codecov Report

❌ Patch coverage is 29.32735% with 788 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.21%. Comparing base (ef31e09) to head (2259e80).
⚠️ Report is 775 commits behind head on main.

Files with missing lines Patch % Lines
...ons/ai-security-guard/lvwang/common/text/openai.go 5.35% 157 Missing and 2 partials ⚠️
...m-go/extensions/ai-security-guard/config/config.go 49.48% 124 Missing and 24 partials ⚠️
...rity-guard/lvwang/multi_modal_guard/text/openai.go 0.00% 139 Missing ⚠️
...urity-guard/lvwang/multi_modal_guard/image/qwen.go 0.00% 80 Missing ⚠️
...ity-guard/lvwang/multi_modal_guard/image/openai.go 0.00% 72 Missing ⚠️
...ai-security-guard/lvwang/common/request_builder.go 71.15% 39 Missing and 6 partials ⚠️
...security-guard/lvwang/multi_modal_guard/handler.go 0.00% 41 Missing ⚠️
...y-guard/lvwang/text_moderation_plus/text/openai.go 46.57% 34 Missing and 5 partials ⚠️
...ugins/wasm-go/extensions/ai-security-guard/main.go 32.50% 26 Missing and 1 partial ⚠️
...urity-guard/lvwang/text_moderation_plus/handler.go 24.00% 19 Missing ⚠️
... and 2 more
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3075      +/-   ##
==========================================
+ Coverage   35.91%   42.21%   +6.30%     
==========================================
  Files          69       94      +25     
  Lines       11576    12061     +485     
==========================================
+ Hits         4157     5092     +935     
+ Misses       7104     6601     -503     
- Partials      315      368      +53
Flag Coverage Δ
wasm-go-plugin-ai-security-guard 30.50% <29.32%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...ity-guard/lvwang/multi_modal_guard/image/common.go 0.00% <0.00%> (ø)
...asm-go/extensions/ai-security-guard/utils/utils.go 47.36% <47.36%> (ø)
...urity-guard/lvwang/text_moderation_plus/handler.go 24.00% <24.00%> (ø)
...ugins/wasm-go/extensions/ai-security-guard/main.go 50.72% <32.50%> (ø)
...y-guard/lvwang/text_moderation_plus/text/openai.go 46.57% <46.57%> (ø)
...security-guard/lvwang/multi_modal_guard/handler.go 0.00% <0.00%> (ø)
...ai-security-guard/lvwang/common/request_builder.go 71.15% <71.15%> (ø)
...ity-guard/lvwang/multi_modal_guard/image/openai.go 0.00% <0.00%> (ø)
...urity-guard/lvwang/multi_modal_guard/image/qwen.go 0.00% <0.00%> (ø)
...rity-guard/lvwang/multi_modal_guard/text/openai.go 0.00% <0.00%> (ø)
... and 2 more

... and 97 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

rinfx added 5 commits October 31, 2025 13:49
@rinfx
update
5061fad 
Change-Id: Ie4289f1a902ed79820df60ed60a43fcf4795ba03
Co-developed-by: Cursor <[email protected]>
@rinfx
update ut
f1011d1 
Change-Id: I1b3f9e4a41cea897206feb78d83ea3d3dda278cf
Co-developed-by: Cursor <[email protected]>
@rinfx
update
e944ba8 
Change-Id: I7af2865922eb48d99c0243b660309af51f7e9d7f
Co-developed-by: Cursor <[email protected]>
@rinfx
update
e02fa66 
Change-Id: Iffbd37b4ae46742036dc86fda7ea591dd14b154e
Co-developed-by: Cursor <[email protected]>
@rinfx
bugfix
2259e80 
Change-Id: Ie93142b5856104f45ea2b1d58077311b58adc1f7
Co-developed-by: Cursor <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@johnlanni johnlanni Awaiting requested review from johnlanni johnlanni is a code owner

@CH3CHO CH3CHO Awaiting requested review from CH3CHO CH3CHO is a code owner

@erasernoob erasernoob Awaiting requested review from erasernoob erasernoob is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rinfx @codecov-commenter